-
-
Notifications
You must be signed in to change notification settings - Fork 785
[12.0] update upstream e7984c4968c #2134
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
pedrobaeza
merged 21 commits into
OCA:12.0
from
ForgeFlow:12.0-update-upstream-e7984c4968c
Dec 18, 2019
Merged
[12.0] update upstream e7984c4968c #2134
pedrobaeza
merged 21 commits into
OCA:12.0
from
ForgeFlow:12.0-update-upstream-e7984c4968c
Dec 18, 2019
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Create a move line from stock to shelf1 with PACK1 - Create a move line from stock to shelf2 with PACK1 - Validate It should be be blocked. The contents of a package should all be in the same location since the pack itself is in a location. It's due to commit 78d6f4607496da06b574c02ab545ffc562a35bb3 that fix a rounding error. However it should check for quants that have quantity since they represents the package content. closes odoo/odoo#38465 Signed-off-by: Arnold Moyaux <[email protected]>
- Create a product A with a BOM - Create a MO for 3000 units of A - Product 3000 units - Unlock the MO and change the Consumed Materials and Finished Products quantity to less than 3000, e.g. 2900 - Update the Quantity To Produce to 2900 The Finished Products move lines are deleted, but the Consumed Materials are left unchanged. This is because decreasing the quantity triggers a `_do_unreserve` which deletes the move lines: https://github.com/odoo/odoo/blob/bf81e38f685abbd4632c7be5cc381dfa38a9b7b8/addons/stock/models/stock_move.py#L399 The issue is similar to df7953f, so we use the same solution: unreserve, but keep track of the quantities. opw-2151927 closes odoo/odoo#41933 Signed-off-by: Nicolas Martinelli (nim) <[email protected]>
The filtered on message_partner_ids should only apply for message_subscribe and not for invoice reference auto-computation closes odoo/odoo#40936 Signed-off-by: Laurent Smet <[email protected]>
Since commit 91560fa, we are using sudo to write calendar_last_notif_ack, but we need to update the real partner stored in the context. opw-2133566 closes odoo/odoo#41947 Signed-off-by: Nicolas Martinelli (nim) <[email protected]>
Purpose of this commit is to better support custom styling used notably for mass mailing and outgoing emails. Indeed styling is whitelisted when storing html. Finding it in html requires some parsing. This commit improves the regex used to find it when semicolons are involved. It allows more use cases to be correctly supported. Task ID #2125856 PR #40433 closes odoo/odoo#41964 X-original-commit: 5235a49 Signed-off-by: Thibault Delavallee (tde) <[email protected]>
The field `standard_price` has the `Product Price` precision. Closes #41729 opw-2154840 closes odoo/odoo#41987 Signed-off-by: Nicolas Martinelli (nim) <[email protected]> Co-authored-by: Yennifer Santiago <[email protected]>
Was added at 958f910 but lost during design change closes odoo/odoo#42053 X-original-commit: 560dba5 Signed-off-by: Martin Trigaux (mat) <[email protected]>
closes odoo/odoo#41959 Signed-off-by: Martin Trigaux (mat) <[email protected]>
A new currency has been introduced in Mauritania in 2018: https://en.wikipedia.org/wiki/Mauritanian_ouguiya opw-2156411 closes odoo/odoo#41986 X-original-commit: 321fc5b Signed-off-by: Nicolas Martinelli (nim) <[email protected]>
Steps to reproduce the bug: - Let's consider a sale included tax T1 of 10% and a sale excluded tax T2 of 0% - Let's consider a product P with T1 and a sale price of 110€ - Let's consider a fiscal position FP that mappes T1 to T2 - Let's consider a customer C with FP as fiscal position - Create a customer invoice for C - Add P on the first line and T1 is replaced by T2 Bug: The unit price of P was still 110€ instead of 100€ because the included tax was not removed from the base price of P. opw:2150564 closes odoo/odoo#41991 Signed-off-by: Simon Goffin (sig) <[email protected]>
opw-2157386 closes odoo/odoo#42035 X-original-commit: e8f080b Signed-off-by: Nicolas Martinelli (nim) <[email protected]>
Not having a tax group, we can have a side effect. It hasn't failed before because the field is required only on screen, but not on DB. closes odoo/odoo#42046 Signed-off-by: Josse Colpaert <[email protected]>
Steps to reproduce: -install website, website_partner, contacts and studio -go to contacts and open any partner -click the studio icon (top-right corner of the screen) -add the 'website_description' field to the partner Previous behavior: the description field is not translatable and attempting to change the "translatable" option in technical > fields spawns an error Current behavior: the description field is translatable opw-2155115 closes odoo/odoo#42023 X-original-commit: 186b599 Signed-off-by: Nicolas Lempereur (nle) <[email protected]>
closes odoo/odoo#42066 Signed-off-by: Jérémy Kersten (jke) <[email protected]>
If a visitor has access to a task through access_token, he should have access to the task attachments. He already see the list of attachment and their name, but since the task access_token is not propagated to the attachment he doesn't have the rights to see them. In this PR, we generate the attachments access_token and provide them to the user that is viewing a task with an access_token. opw-2125252 closes #41881 Signed-off-by: Nicolas Lempereur (nle) <[email protected]>
When running multiple odoo on the same machine (in order to test multiple things concurrently, etc...) it's easily enough to start them on 8069, 8070, 8071, ... and that *seems* to work, but then they start walking on one another and *losing* chromes entirely, so you end up with hundreds of chrome processes: right now with 2 Odoo running tests my machine is at 365 Chrome processes, 550 process and 3180 threads total (should be ~160 and ~700 with 2 chrome processes at most). Instead, ask the OS to ask for a devtools port, then close the socket and pass that to the child process. There's a race of sort for the instant between closing the socket and Chrome reopening it, but: * the window is very short * we got a random port from the ephemeral range, it's possible somebody else gets it inbetween but unlikely (the 50% on a birthday attack is 200 for Linux's ephemeral range, it's a somewhat lower 150 for the IANA range used by BSDs and Windows) Sadly chrome doesn't stop if it can't bind to the port it's given. closes odoo/odoo#42071 Signed-off-by: Christophe Monniez (moc) <[email protected]>
Before this commit, the log displayed "set as done" before actually trying to put the transaction in done. If an error then occurred in the'_set_transaction_done' function and the transaction could not be processed, this could lead to a misinterpretation of these logs. This log is therefore moved after the transaction is processed and is only displayed if it is actually successful. This is valid for'done','pending' and'cancel'. closes odoo/odoo#39364 Signed-off-by: Richard Mathot (rim) <[email protected]>
Before this fix, when a refresh of the google token was triggered and error 400 where returned, if the user did not have write rights on res_users, an access rights was raised Now, the write is done and the real error is raised for the client. Part of OPW-2123045 (needed to further investigations) closes odoo/odoo#41754 X-original-commit: b4f794c Signed-off-by: Richard Mathot (rim) <[email protected]>
…984c4968c No conflicts
This comment has been minimized.
This comment has been minimized.
pedrobaeza
reviewed
Dec 18, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update to include fix in
l10n_es.--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr